Otel collector integration by M4itee · Pull Request #519 · netdata/helmchart

M4itee · 2026-01-23T15:31:10Z

Adding ability to get otel metrics from the kubernetes

CLAassistant · 2026-01-23T15:31:20Z

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

Mateusz seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

charts/netdata/Chart.yaml

Copilot

Pull request overview

This pull request adds OpenTelemetry Collector integration to the Netdata Helm chart, enabling the collection of logs from Kubernetes pods and forwarding them to a netdata instance via OTLP protocol.

Changes:

Added netdataOpentelemetry deployment and service for receiving OTLP data
Integrated opentelemetry-collector as a subchart dependency for collecting Kubernetes logs
Fixed YAML indentation issues in ingress hosts and child tolerations configurations

Reviewed changes

Copilot reviewed 8 out of 10 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
charts/netdata/values.yaml	Added netdataOpentelemetry configuration section and otel-collector subchart settings; fixed indentation issues
charts/netdata/templates/netdata-otel/service.yaml	Created service to expose OTLP endpoint on port 4317
charts/netdata/templates/netdata-otel/secrets.yaml	Created secrets template for OTEL configuration
charts/netdata/templates/netdata-otel/persistentvolumeclaim.yaml	Created PVC for OTEL log storage
charts/netdata/templates/netdata-otel/deployment.yaml	Created deployment for netdata instance with OTEL plugin enabled
charts/netdata/templates/netdata-otel/configmap.yaml	Created configmap template for OTEL configuration
charts/netdata/templates/_helpers.tpl	Added helper templates for netdataOpentelemetry configs management
charts/netdata/Chart.yaml	Added opentelemetry-collector subchart dependency
charts/netdata/Chart.lock	Generated lock file for subchart dependency
charts/netdata/charts/opentelemetry-collector-0.115.0.tgz	Added OpenTelemetry Collector subchart package

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

charts/netdata/values.yaml

ilyam8 · 2026-01-24T20:49:01Z

@cubic-dev-ai review this PR

Copilot

Pull request overview

Copilot reviewed 8 out of 10 changed files in this pull request and generated 5 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

charts/netdata/values.yaml

charts/netdata/templates/netdata-otel/deployment.yaml

charts/netdata/values.yaml

charts/netdata/templates/netdata-otel/deployment.yaml

witalisoft

make sure that otel is also enabled when we are running checks

charts/netdata/templates/netdata-otel/deployment.yaml

charts/netdata/values.yaml

M4itee · 2026-01-30T14:30:55Z

templates/netdata-README.md.gotmpl

+
+The following table lists the configurable parameters of the netdata chart and their default values.
+
+{{ template "chart.valuesTableHtml" . }}


HTML because it renders better the multiline comments and values + adds some extra feats like copy, colors. Any markdown parser will render it no problem anyway.

M4itee · 2026-01-30T14:41:19Z

.github/workflows/checks.yml

      - name: Run chart-testing (install)
        run: ct install --target-branch ${{ github.event.repository.default_branch }}
+
+      - name: Run chart-testing (install with OpenTelemetry)


an extra for easier problem catching, in theory we can just use this one and delete the regular install. Good thing is that at the end of the ct install there is a cleanup so they are both independent.

M4itee · 2026-01-30T14:58:34Z

Last thing left for now is to make docs a bit more readable, after that it should be ready for review

Copilot

Pull request overview

Copilot reviewed 12 out of 14 changed files in this pull request and generated 4 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

charts/netdata/templates/netdata-otel/deployment.yaml

charts/netdata/values.yaml

.github/workflows/checks.yml

install steps in cheks now wait for resources, archive is no longer in the branch - it is pulled during the tests and release.

Copilot

Pull request overview

Copilot reviewed 13 out of 14 changed files in this pull request and generated 2 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-02-23T13:02:51Z

charts/netdata/values.yaml

+# OpenTelemetry Collector subchart configuration
+# This is an optional component that allows to gather logs from k8s cluster (in this configuration).
+# If you already have an exporter of any kind, just point it to the netdata-otel service
+# Documentation: https://opentelemetry.io/docs/platforms/kubernetes/helm/collector/


The PR description says it adds the ability to get OpenTelemetry metrics from Kubernetes, but the otel-collector default config here only defines a logs pipeline (no metrics pipeline/receivers). Either update the PR description to mention logs collection, or extend the collector configuration to actually collect/export metrics as well.

Copilot · 2026-02-23T13:02:51Z

charts/netdata/values.yaml

+      path: /etc/netdata/otel.yaml
+      data: |
+        endpoint:
+          path: "0.0.0.0:4317"


netdataOpentelemetry.service.port is configurable and used by the Service/Deployment, but the generated otel.yaml config hard-codes the listener as 0.0.0.0:4317. If a user changes the service/container port, Netdata will still listen on 4317 and OTLP traffic will fail. Make the endpoint.path value reference .Values.netdataOpentelemetry.service.port (and keep it rendered via tpl) so the config stays consistent with the Service port.

Suggested change

path: "0.0.0.0:4317"

path: {{ tpl (printf "\"0.0.0.0:%v\"" .Values.netdataOpentelemetry.service.port) . }}

@M4itee, should we fix this?

Honestly I'm fine either way. I think it makes it touch less readable but automates the whole thing. Also I'm not sure if it will work this way correctly (I think printf alone will work) although I'm also not sure if it will work correctly though it will be easy enough to test - will it have a problem with delclaration as a number vs a as string of the port number. I'll leave decision up to you @ilyam8 .

if this is true

If a user changes the service/container port, Netdata will still listen on 4317 and OTLP traffic will fail.

we should fix this

ok I checked what we are doing with the parent port. I will fix it, it should be the same.

charts/netdata/Chart.lock

Copilot

Pull request overview

Copilot reviewed 13 out of 14 changed files in this pull request and generated 3 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

charts/netdata/templates/netdata-otel/deployment.yaml

Copilot · 2026-02-23T13:12:37Z

templates/netdata-README.md.gotmpl

+
+Disabled by default:
+
+- A Netdata restarter `CronJob`. Its main purpose is to automatically update Netdata when using nightly releases.


The README template should document the new OpenTelemetry integration features (netdataOpentelemetry deployment and otel-collector subchart) in the "Disabled by default" section, similar to how the restarter CronJob is documented. This would help users discover these features and understand what optional components are available.

Suggested change

- A Netdata restarter `CronJob`. Its main purpose is to automatically update Netdata when using nightly releases.

- A Netdata restarter `CronJob`. Its main purpose is to automatically update Netdata when using nightly releases.

- A Netdata OpenTelemetry collector `Deployment` (`netdataOpentelemetry`) for exposing Netdata metrics via OTLP to external observability backends.

- An optional `otel-collector` subchart that deploys a configurable OpenTelemetry Collector instance for advanced pipelines and exporting data to third-party systems.

charts/netdata/values.yaml

Copilot

Pull request overview

Copilot reviewed 13 out of 14 changed files in this pull request and generated no new comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

ilyam8 requested a review from Copilot January 24, 2026 20:38

Copilot started reviewing on behalf of ilyam8 January 24, 2026 20:39 View session

ilyam8 reviewed Jan 24, 2026

View reviewed changes

charts/netdata/Chart.yaml Outdated Show resolved Hide resolved

Copilot AI reviewed Jan 24, 2026

View reviewed changes

charts/netdata/values.yaml Show resolved Hide resolved

charts/netdata/values.yaml Outdated Show resolved Hide resolved

charts/netdata/values.yaml Show resolved Hide resolved

ilyam8 requested a review from Copilot January 24, 2026 20:51

Copilot started reviewing on behalf of ilyam8 January 24, 2026 20:52 View session

Copilot AI reviewed Jan 24, 2026

View reviewed changes

witalisoft reviewed Jan 26, 2026

View reviewed changes

charts/netdata/templates/netdata-otel/deployment.yaml Outdated Show resolved Hide resolved

charts/netdata/values.yaml Show resolved Hide resolved

M4itee force-pushed the otel-collector-integration branch 3 times, most recently from 5c3757b to 4b1dbb2 Compare January 30, 2026 14:28

M4itee commented Jan 30, 2026

View reviewed changes

M4itee force-pushed the otel-collector-integration branch 5 times, most recently from 73258ee to 138f84e Compare January 30, 2026 17:16

ilyam8 requested a review from Copilot January 30, 2026 18:00

Copilot started reviewing on behalf of ilyam8 January 30, 2026 18:00 View session

Copilot AI reviewed Jan 30, 2026

View reviewed changes

charts/netdata/templates/netdata-otel/deployment.yaml Show resolved Hide resolved

charts/netdata/values.yaml Outdated Show resolved Hide resolved

charts/netdata/values.yaml Outdated Show resolved Hide resolved

.github/workflows/checks.yml Show resolved Hide resolved

M4itee force-pushed the otel-collector-integration branch 6 times, most recently from abfbb32 to 8705505 Compare February 2, 2026 12:35

Mateusz added 9 commits February 23, 2026 12:50

changes to docs, checks, otel values

1bb92a1

fixing deployment template for otel

459f125

Removing debug log level from otel, testing doc checks

871a491

Updating documentation

2948c33

values table in non-html

a8933a7

removing long desc from some values

7463980

changes to workflows - release and checks

6451bc0

install steps in cheks now wait for resources, archive is no longer in the branch - it is pulled during the tests and release.

adding security context

daf1396

removing helm install step from release (not needed)

f7bb9e2

M4itee force-pushed the otel-collector-integration branch from 8760d58 to f7bb9e2 Compare February 23, 2026 11:50

updates to docs

66fd83b

ilyam8 requested a review from Copilot February 23, 2026 12:53

Copilot started reviewing on behalf of ilyam8 February 23, 2026 12:53 View session

Copilot AI reviewed Feb 23, 2026

View reviewed changes

ilyam8 reviewed Feb 23, 2026

View reviewed changes

charts/netdata/Chart.lock Show resolved Hide resolved

ilyam8 requested a review from Copilot February 23, 2026 13:03

Copilot started reviewing on behalf of ilyam8 February 23, 2026 13:04 View session

Copilot AI reviewed Feb 23, 2026

View reviewed changes

ilyam8 requested review from Copilot and ilyam8 February 23, 2026 13:14

Copilot started reviewing on behalf of ilyam8 February 23, 2026 13:15 View session

Copilot AI reviewed Feb 23, 2026

View reviewed changes

thiagoftsm previously approved these changes Feb 23, 2026

View reviewed changes

witalisoft previously approved these changes Feb 24, 2026

View reviewed changes

streamlining port change for netdataopentelemetry

77344a0

M4itee dismissed stale reviews from witalisoft and thiagoftsm via 77344a0 February 25, 2026 11:15

ilyam8 approved these changes Feb 25, 2026

View reviewed changes

M4itee merged commit cc418a7 into master Feb 25, 2026
2 of 3 checks passed

M4itee deleted the otel-collector-integration branch February 25, 2026 11:59


		The following table lists the configurable parameters of the netdata chart and their default values.

		{{ template "chart.valuesTableHtml" . }}

	path: "0.0.0.0:4317"
	path: {{ tpl (printf "\"0.0.0.0:%v\"" .Values.netdataOpentelemetry.service.port) . }}


		Disabled by default:

		- A Netdata restarter `CronJob`. Its main purpose is to automatically update Netdata when using nightly releases.

Conversation

M4itee commented Jan 23, 2026

Uh oh!

CLAassistant commented Jan 23, 2026

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

ilyam8 commented Jan 24, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

witalisoft left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

M4itee Jan 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

M4itee Jan 30, 2026

Choose a reason for hiding this comment

Uh oh!

M4itee commented Jan 30, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Feb 23, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 23, 2026

Choose a reason for hiding this comment

Uh oh!

ilyam8 Feb 24, 2026

Choose a reason for hiding this comment

Uh oh!

M4itee Feb 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

ilyam8 Feb 25, 2026

Choose a reason for hiding this comment

Uh oh!

M4itee Feb 25, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Copilot AI Feb 23, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

M4itee Jan 30, 2026 •

edited

Loading

M4itee Feb 25, 2026 •

edited

Loading